Skip to content

[FIX] 구매자 - 참여 리스트 모집완료 상태 분철글 상태를 따르게#200

Merged
88guri merged 2 commits intodevelopfrom
fix/#199/participation-list-change-status
Jan 23, 2026
Merged

[FIX] 구매자 - 참여 리스트 모집완료 상태 분철글 상태를 따르게#200
88guri merged 2 commits intodevelopfrom
fix/#199/participation-list-change-status

Conversation

@88guri
Copy link
Copy Markdown
Collaborator

@88guri 88guri commented Jan 23, 2026 •
edited by coderabbitai bot
Loading

📌 관련 이슈

✨ 변경 사항

  • 그냥 OrderStatus 따라서 COMPLETED 뜨던걸, GroupBuyPostStatus가 RECRUTING이면 그걸 우선적으로 따르도록 변경했습니닼

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

  • 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
  • 로컬에서 빌드 및 실행이 정상적으로 되나요?
  • 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
  • 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 배송 완료 상태 확인 검증 강화로 부정확한 작업 방지
    • 주문 상태 매핑 로직 개선으로 사용자에게 표시되는 상태 정확도 향상
    • 배송 완료 처리 시 남은 미배송 주문 없음을 확인하는 제어 흐름 추가

✏️ Tip: You can customize this high-level summary in your review settings.

@88guri 88guri requested a review from PBEM22 January 23, 2026 11:41
@88guri 88guri self-assigned this Jan 23, 2026
@88guri 88guri added 🚨 Fix 버그 수정 🐵 시현 시현이에요 ✋Participation 공구 참여, 관리 labels Jan 23, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 23, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

ParticipationService에서 배송 완료 처리 시 SHIPPED 상태 검증을 강제하고, 게시글 상태와 주문 상태를 모두 고려하여 클라이언트에 반환할 상태를 결정하는 이중 매개변수 메서드로 변경했습니다. RECRUITING 상태에 우선순위를 부여하는 로직이 추가되었습니다.

Changes

Cohort / File(s) 설명
배송 완료 및 상태 매핑 로직 개선
src/main/java/.../participation/service/ParticipationService.java		 SHIPPED 상태 확인 추가, 게시글 상태(GroupBuyPostStatus)와 주문 상태(OrderStatus)를 모두 받는 mapToClientPostStatus 메서드로 변경. RECRUITING 상태 우선순위 로직 도입. 마지막 배송 대기 주문 완료 시 post delivery 처리 추가

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Possibly related PRs

Suggested labels

🔥 P0

Suggested reviewers

  • PBEM22
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 반영하고 있습니다. 구매자 참여 리스트에서 모집완료 상태가 분철글(그룹바이 포스트) 상태를 따르도록 변경한 내용을 정확히 설명합니다.
Linked Issues check ✅ Passed 코드 변경사항이 #199 이슈의 요구사항을 충족합니다. GroupBuyPostStatus에 RECRUITING 우선순위를 부여하여 구매자 참여 리스트에서 포스트 레벨의 모집 상태를 정확하게 반영하도록 구현했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #199 이슈의 범위 내에 있습니다. mapToClientPostStatus 메서드 시그니처 업데이트와 상태 매핑 로직 개선, 그리고 배송 완료 검증 추가가 모두 모집완료 상태 개선이라는 목표에 부합합니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

PBEM22
PBEM22 approved these changes Jan 23, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@PBEM22 PBEM22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이번엔 진자 믿습니다.

@88guri 88guri merged commit 7da1e70 into develop Jan 23, 2026
2 checks passed
@88guri 88guri deleted the fix/#199/participation-list-change-status branch January 23, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PBEM22 PBEM22 PBEM22 approved these changes

Assignees

@88guri 88guri

Labels

🚨 Fix 버그 수정 ✋Participation 공구 참여, 관리 🐵 시현 시현이에요

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FIX] 구매자 - 참여 리스트 모집완료 상태 분철글 상태를 따르게

2 participants

@88guri @PBEM22